fix: we should unspend transaction tx outputs when the tx spending it…#281
Conversation
f72d69f to
89fdbe1
Compare
1aa8524 to
0e09722
Compare
0e09722 to
865a58f
Compare
deddf22 to
daca065
Compare
daca065 to
c042b64
Compare
2211677 to
e35c21b
Compare
ec026c3 to
ce6a19c
Compare
|
e6458b8 fixes an issue with the integration tests causing them to consistently fail when running all tests at once but to succeed when ran individually, here is a better explanation: Problem: The Root Cause:
Solution: Changed from shared context to dynamically created LRU cache per machine instance:
More context on the This guard is a performance optimization that prevents redundant database operations by checking if transaction metadata has actually changed. It compares the cached hash of a transaction's metadata against the current event's metadata hash. If they match, the event is ignored since we've already processed this exact state. The guard is designed to skip repeated |
e6458b8 to
cd1c869
Compare
ec031dc to
7016f91
Compare
Acceptance Criteria
txproposalin case thistxproposalfails.walletbalances when a transaction is voidedIMPORTANT: In this PR we are (1) using a experimental docker image for the new tests and (2) skipping genesis address balance check
(1) This will be fixed when HathorNetwork/hathor-core#1392 is merged
(2) This is a race condition, tests pass when running individually, we must debug and fix it in a later PR
Checklist
master, confirm this code is production-ready and can be included in future releases as soon as it gets merged